Skip to content

Comments

Add OPS/ prefix support and deduplicate prefix list#26

Merged
inkroom merged 1 commit intoinkroom:masterfrom
dee:fix-ops
Feb 24, 2026
Merged

Add OPS/ prefix support and deduplicate prefix list#26
inkroom merged 1 commit intoinkroom:masterfrom
dee:fix-ops

Conversation

@dee
Copy link
Contributor

@dee dee commented Feb 6, 2026

Some EPUB files store content under "OPS/" instead of OEBPS/EPUB paths.
This adds "OPS/" to the supported prefixes and centralizes the prefix list.

@dee
Copy link
Contributor Author

dee commented Feb 10, 2026

CI fails because tests run with cargo test --workspace,
which compiles the iepub-derive crate.

The failure comes from duplicate set_k generated by
option_string_method!(info,k) and option_string_method!(k).

This seems unrelated to the OPS prefix change.

@michael-eddy
Copy link
Contributor

CI fails because tests run with cargo test --workspace, which compiles the iepub-derive crate.

The failure comes from duplicate set_k generated by option_string_method!(info,k) and option_string_method!(k).

This seems unrelated to the OPS prefix change.

now is chinese new year, maybe the cratetor in busy 😄, be better upload the demo epub for testing feture

@inkroom inkroom merged commit e77c337 into inkroom:master Feb 24, 2026
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants